Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Testing #86

Merged
merged 42 commits into from
Apr 25, 2023
Merged

Testing #86

merged 42 commits into from
Apr 25, 2023

Conversation

ericgaspar
Copy link
Member

Problem

  • Description of why you made this PR

Solution

  • And how do you fix that problem

PR Status

  • Code finished and ready to be reviewed/tested
  • The fix/enhancement were manually tested (if applicable)

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

@ericgaspar
Copy link
Member Author

!testme

@yunohost-bot
Copy link
Contributor

Alrighty!
Test Badge

@Gredin67 Gredin67 mentioned this pull request Nov 17, 2022
Open
@Gredin67
Copy link
Collaborator

Install successes (after a previous remove, see #87
But I get

Warning: ds:docservice: ERROR (no such process)
Warning: ds:docservice: ERROR (no such process)
Warning: ds:converter: ERROR (no such process)
Warning: ds:converter: ERROR (no such process)

@Gredin67
Copy link
Collaborator

@zamentur could you review this PR?

@Gredin67 Gredin67 requested a review from zamentur December 19, 2022 15:33
@Gredin67
Copy link
Collaborator

But I get

Warning: ds:docservice: ERROR (no such process)
Warning: ds:docservice: ERROR (no such process)
Warning: ds:converter: ERROR (no such process)
Warning: ds:converter: ERROR (no such process)

This can be manually solved by #84 and #88 or automatically with YunoHost-Apps/nextcloud_ynh#537

@Gredin67
Copy link
Collaborator

@ericgaspar I incidentally pushed to the Testing branch some changes in the README. b2b18e1
And I realized that the auto-updater erased it. How can I get the changes merged?

@ericgaspar
Copy link
Member Author

@Gredin67 The README.md and README_fr.md files are generated automatically from the DESCRIPTION.md, DISCLAIMER.md, DESCRIPTION_fr.md and DISCLAIMER_fr.md files. If you want to modify something in the README, you must modify these files.

@ericgaspar
Copy link
Member Author

!testme

@yunohost-bot
Copy link
Contributor

Meow 🐈
Test Badge

@Gredin67
Copy link
Collaborator

!testme

@yunohost-bot
Copy link
Contributor

😜
Test Badge

@Gredin67
Copy link
Collaborator

!testme

@ericgaspar
Copy link
Member Author

!testme

@yunohost-bot
Copy link
Contributor

May the CI gods be with you!
Test Badge

@ericgaspar
Copy link
Member Author

!testme

@yunohost-bot
Copy link
Contributor

Alrighty!
Test Badge

@ericgaspar
Copy link
Member Author

!testme

@yunohost-bot
Copy link
Contributor

Alrighty!
Test Badge

@ericgaspar
Copy link
Member Author

!testme

@yunohost-bot
Copy link
Contributor

✌️
Test Badge

@ericgaspar ericgaspar merged commit 205cc12 into master Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zamentur zamentur Awaiting requested review from zamentur

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@ericgaspar @yunohost-bot @Gredin67 @Typhlos @zamentur @tituspijean @sebmennetrier